repo: detached sigs: Use error prefixing instead of overwriting

Noted when "rpm-ostree compose sign" failed to write to a repo due to
permissions.

diff --git a/src/libostree/ostree-repo-commit.c b/src/libostree/ostree-repo-commit.c
index 97ff745389c99852e91386bab6da6acade5906a4..9d8bf8bd102a6019b09130004a58787aae031eb9 100644 (file)
--- a/src/libostree/ostree-repo-commit.c
+++ b/src/libostree/ostree-repo-commit.c
@@ -1925,6 +1925,7 @@ ostree_repo_read_commit_detached_metadata (OstreeRepo      *self,
         }
       else
         {
+          g_prefix_error (error, "Unable to read existing detached metadata: ");
           g_propagate_error (error, temp_error);
           goto out;
         }
@@ -1971,7 +1972,10 @@ ostree_repo_write_commit_detached_metadata (OstreeRepo      *self,
                                 g_variant_get_size (normalized),
                                 NULL, FALSE, 0, NULL,
                                 cancellable, error))
-    goto out;
+    {
+      g_prefix_error (error, "Unable to write detached metadata: ");
+      goto out;
+    }
 
   ret = TRUE;
  out:

diff --git a/src/libostree/ostree-repo.c b/src/libostree/ostree-repo.c
index 32524d22a46c635d84b1cfc1ad171f4d881d79e4..a05d481d73271ec4f82a5186535321a4ede23919 100644 (file)
--- a/src/libostree/ostree-repo.c
+++ b/src/libostree/ostree-repo.c
@@ -2995,11 +2995,7 @@ ostree_repo_append_gpg_signature (OstreeRepo     *self,
                                                   &metadata,
                                                   cancellable,
                                                   error))
-    {
-      g_set_error (error, G_IO_ERROR, G_IO_ERROR_FAILED,
-                   "Unable to read existing detached metadata");
-      goto out;
-    }
+    goto out;
 
   new_metadata = _ostree_detached_metadata_append_gpg_sig (metadata, signature_bytes);
 
@@ -3008,11 +3004,7 @@ ostree_repo_append_gpg_signature (OstreeRepo     *self,
                                                    new_metadata,
                                                    cancellable,
                                                    error))
-    {
-      g_set_error (error, G_IO_ERROR, G_IO_ERROR_FAILED,
-                   "Unable to read existing detached metadata");
-      goto out;
-    }
+    goto out;
 
   ret = TRUE;
  out: